-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates to MoNuSeg #157
Updates to MoNuSeg #157
Conversation
anwai98
commented
Oct 18, 2023
- Added processing for the monuseg inputs (including xml to numpy array conversion) (kept the expected directory formats consistent)
- (WIP) Add the test split
- (TODO) Check the organ splits from the challenge's metadata and make those splits (mentioned in TODOs)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @anwai98 ,
not sure if this is ready to review yet, but I had a quick look. In addition to the comments in the code, I tried the download, but it fails with
Access denied with the following error:
Cannot retrieve the public link of the file. You may need to change
the permission to 'Anyone with the link', or have had many accesses.
You may still be able to access the file from the browser:
https://drive.google.com/uc?export=download&id=1ZgqFJomqQGNnsx7w7QBzQQMVA16lbVCA
Hmm, pretty interesting. I just tried it again and the download works for me. Is it related to the
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good in principle, but do you still want to implement the organ selection before we merge it?
Yes, I wanted to complete the setup, hence took it back as a draft. Would take care of it and put it back to review later. Thanks. |